Add __repr__ methods to core Qiskit classes#16148
Add __repr__ methods to core Qiskit classes#16148yhwang wants to merge 4 commits intoQiskit:mainfrom
Conversation
Implements __repr__ for Gate, ControlledGate, QuantumCircuit, PassManager, StatevectorSampler, and StatevectorEstimator. All implementations use the non-eval-able <ClassName ...> format as discussed in issue Qiskit#8594, providing human-readable debugging information without attempting to be eval-able. Includes comprehensive unit tests (26 tests) and release notes. Fixes Qiskit#8594 Co-authored-by: Richard Rodenbusch <rrodenbusch@gmail.com> Signed-off-by: Yihong Wang <yh.wang@ibm.com>
|
Thank you for opening a new pull request. Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient. While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone. One or more of the following people are relevant to this code:
|
This comment was marked as low quality.
This comment was marked as low quality.
Coverage Report for CI Build 25509847910Warning Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes. Coverage increased (+0.009%) to 87.627%Details
Uncovered ChangesNo uncovered changes found. Coverage Regressions10 previously-covered lines in 2 files lost coverage.
Coverage Stats
💛 - Coveralls |
Implements repr for Gate, ControlledGate, QuantumCircuit, PassManager, StatevectorSampler, and StatevectorEstimator.
All implementations use the non-eval-able <ClassName ...> format as discussed in issue #8594, providing human-readable debugging information without attempting to be eval-able.
Includes comprehensive unit tests (26 tests) and release notes.
One thing worth mentioning is that the changes in this PR are derived from this closed PR. Therefore,
Richard Rodenbuschis added as the co-author of the commit.Fixes #8594
AI/LLM disclosure